Skip to content

Remove inbound command and inbox provider surface#56

Merged
qasim-nylas merged 1 commit intonylas:mainfrom
qasim-nylas:qasim/remove-inbound-surface
Apr 17, 2026
Merged

Remove inbound command and inbox provider surface#56
qasim-nylas merged 1 commit intonylas:mainfrom
qasim-nylas:qasim/remove-inbound-surface

Conversation

@qasim-nylas
Copy link
Copy Markdown
Collaborator

Summary

  • remove the deprecated inbound command, docs, adapters, TUI/UI surfaces, and provider=inbox domain plumbing
  • hide and reject legacy inbox connectors across auth/admin CLI paths, including direct connector show by ID
  • wire focused inbound/provider removal and agent checks into make ci-full

Testing

  • go test ./internal/cli/admin
  • go build -o bin/nylas ./cmd/nylas
  • go test ./internal/cli/integration/... -tags=integration -run 'TestCLI_(ConnectorSurfaces_HideInboxProvider|AdminConnectorsCreate_RejectsInboxProvider|AdminConnectorsShow_HidesInboxProvider)$' -v
  • make test-cli-regressions
  • make test-integration-agent
  • make ci-full

@qasim-nylas qasim-nylas requested review from cgeene and mqasimca and removed request for cgeene April 17, 2026 01:02
@qasim-nylas qasim-nylas merged commit b4c429c into nylas:main Apr 17, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants